Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[9.0][MIG][mass_mailing_partner] #91

Closed
wants to merge 1 commit into from
Closed

[9.0][MIG][mass_mailing_partner] #91

wants to merge 1 commit into from

Conversation

bringsvor
Copy link

Just changed the view to get the right button.

@pedrobaeza
Copy link
Member

Can you please extract and move this to the new OCA/social repo that fits better for mass mailing stuff?

@bringsvor
Copy link
Author

So you want me to move the whole module to the OCA/social? I'm a bit short on git skills, but would it be OK to delete it from here and add it to OCA/social? Or is there a smarter way that would preserve history?

@coveralls
Copy link

coveralls commented Apr 26, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 93.59% when pulling 46d4ce7 on bringsvor:9.0 into 1ed22fb on OCA:9.0.

@pedrobaeza
Copy link
Member

Yes, please. We create it that repo after the merge of this module, so for 8.0 we didn't want to move it for keeping stability. But now for 9.0 we can do it. You can see a git guide for extracting correctly the commit history here: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-9.0

rafaelbn
rafaelbn requested changes Oct 4, 2016
View reviewed changes
Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please close this PR and open it again in https://github.com/OCA/social
Thanks

@rafaelbn rafaelbn changed the title Updated to install on Odoo 9.0. [9.0][MIG][mass_mailing_partner] Oct 4, 2016
@rafaelbn
Copy link
Member

rafaelbn commented Oct 4, 2016

Hi @bringsvor are you going to continue with this PR? Let me know please, thanks,

@rafaelbn rafaelbn mentioned this pull request Oct 4, 2016
Closed
@rafaelbn rafaelbn added this to the 9.0 milestone Oct 4, 2016
@pedrobaeza
Copy link
Member

No news, so closing.

@pedrobaeza pedrobaeza closed this Oct 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rafaelbn rafaelbn rafaelbn requested changes

Assignees
No one assigned
Labels
duplicate
Projects
None yet
Milestone
9.0
Development

Successfully merging this pull request may close these issues.
4 participants
@bringsvor @pedrobaeza @coveralls @rafaelbn